Adapt bag of members #107
Adapt bag of members #107
Conversation
|
これをすることでカーネル密度推定において重み付けをすることもできるようになる |
|
@takuro-Ishida @senoura そういえばチュートリアルコードがまともに動くか検証して欲しい！ |
|
リアクション遅れてすみません。#108 のreview含めてなんですが12/16まで特にNC原稿に時間がかかるのでreview遅くなる予定です。ほんとすみません… |
|
@senoura いえいえ、めっちゃ急ぎというわけでもないので大丈夫です！ |
|
@takuro-Ishida @senoura |
|
fit_TSOM_plus_SOM.pyがbug of membersのデータ形式で動いているかを確認すればいいんですかね？ |
|
conflict解消 |
|
@senoura |
|
@senoura あ、テストコードざっくり読んで、テストの内容を把握することはお願いしたい |
|
こういうテストしてんのねハイハイみたいな |
There was a problem hiding this comment.
tests/plus_TSOM/test_plusTSOM.py に関して
メンバー特徴量、SOMおよびTSOMのハイパーパラメータ、メンバーのチーム所属情報を引数にした＋型TSOMを渡辺さん石田さん各自で作りその結果が一致することを確認しました。
チーム所属情報を各チームごとに保有選手の番号をarray形式で保持させた場合と、各メンバーに対してのone hot 表記した場合、両方とも＋型TSOMの結果が一致することを確認しました。
チュートリアルに関して
from libs.models.TSOMPlusSOM import TSOMPlusSOMの箇所で
TSOMPlusSOMってファイルが無いと言われたので同じ階層にあったtsom_plus_som.pyで実行。
チームマップが描画されることを確認しました。
approveします。
|
@senoura コード修正したところあればそちらpushしてもらえませんか？ |
|
|
|
|
|
Approveします！ |
|
せっかくしてくれたApproveを消してしまった... |
|
自分がバグの修正と観測データを描画するところをいじってしまったために @senoura がApproveを消してしまいました. 非常に申し訳ないのですが，別の方にReviewを依頼していただけることってできないですか？ |
ae14watanabe
requested review from
senoura and
TetraMiyazaki
and removed request for
senoura
|
@TetraMiyazaki にお願いしてみたけど、何のこっちゃやろうから取り掛かる時はZOOMで話しましょっ |
|
自分の研究もあるので，優先度的に手をつけられるのは，1ヶ月先とかになるかもしれないです |
|
そこまで優先度は高くないです。にしてもプラス型にカスってるメンバーがほとんど居ないのが辛い所ですね…💦 |
Description 説明
TSOMPlusSOMで入力するgroupの構成メンバーを表現するgroup_featuresにおいて、bag of members形式（グループ数 x 総メンバー数の行列）で与えられるように変更
#104 のコミットをマージしてしまったので、マージは #104 がマージされた後にしたいですー。
** Type of change 変更の種類**
Please delete options that are not relevant for your post.
関係のないオプションは削除してください
How Has This Been Tested? どのようにテストしたか？
テストに新しいメソッドを追加して実行。従来のデータ構造で入れた結果とbag of membersで入れた結果が一致することを確認